Skip to content

Fix uv venv errors because of preexisting environment#17

Merged
olehermanse merged 2 commits intocfengine:mainfrom
victormlg:venv-bug
Mar 5, 2026
Merged

Fix uv venv errors because of preexisting environment#17
olehermanse merged 2 commits intocfengine:mainfrom
victormlg:venv-bug

Conversation

@victormlg
Copy link
Contributor

No description provided.

Signed-off-by: Victor Moene <victor.moene@northern.tech>
@victormlg victormlg force-pushed the venv-bug branch 2 times, most recently from 39ee8ed to ebd63fe Compare March 3, 2026 14:59
@victormlg
Copy link
Contributor Author

The tests pass when changing lint.yaml

on:
  push:
    branches: [main]
  pull_request:
    branches: [main]

to

on: [push, pull_request]

python -m pip install uv
- name: Setup
run: |
uv venv
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@victormlg Did you try adding --clear as the error message suggests?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@victormlg AFAICT this is editing the wrong file. lint has been green, the one that needed help was test.yml.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverted and fixed in #19

@olehermanse olehermanse merged commit d892c14 into cfengine:main Mar 5, 2026
1 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants